Skip to content

fix(desktop): include Chrome app text in macOS SnapShots - #11325

Open
shivamhwp wants to merge 1 commit into
fix/client-device-ticket-http-groupfrom
fix/desktop-chrome-snapshot-title
Open

fix(desktop): include Chrome app text in macOS SnapShots#11325
shivamhwp wants to merge 1 commit into
fix/client-device-ticket-http-groupfrom
fix/desktop-chrome-snapshot-title

Conversation

@shivamhwp

Copy link
Copy Markdown
Collaborator

Chrome SnapShots on macOS can omit app text because CoreGraphics reports the page title while accessibility appends - Google Chrome – Profile. The matcher required the two titles to be identical.

Allow that suffix for macOS Chrome and Chromium bundle IDs after checking for exact matches. Keep process scoping and bounds checks, and reject ambiguous suffix matches even if one window reports itself active. Other platforms and applications keep exact matching.

Fixes #10896. Stacked on #11298. Internal refinement remains pending after the final Fable retry failed.

Validation: 96 focused tests pass across the matcher, accessibility reader, worker lifecycle and macOS capture controls. Six reported-title variants fail against the parent implementation. Desktop typecheck and scoped lint pass. The reader test returns accessibility context for the reported title pair and rejects other processes/applications. Native macOS Chrome verification remains pending; Edge, Brave and Arc are outside this fix. SnapShots are desktop-only; retained context uses existing contracts for local and remote delivery.

Model: GPT-6. Harness: Codex.

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:M 30-99 changed lines (additions + deletions). labels Sep 11, 2026
@macroscopeapp

macroscopeapp Bot commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at 5dcefb7

Macroscope's review found this PR approvable — The production change is narrowly confined to matching macOS Chrome/Chromium accessibility windows whose titles include browser profile suffixes, while preserving existing exact-match and bounds safeguards. Focused tests cover supported variants, ambiguity, platform restrictions, and unrelated titles.

You can add or adjust custom eligibility rules. Learn more.

@github-actions

Copy link
Copy Markdown
Contributor

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ No successful main baseline artifact is available yet. This run establishes the initial measurement.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire 13.6 KiB 15.1 KiB
Codex Thread snapshot wire 7.0 KiB 7.3 KiB
Codex Live turn WebSocket wire 6.6 KiB 7.8 KiB
Codex Live turn WebSocket decoded 57.1 KiB 66.4 KiB
Codex Live turn messages 10 21
Claude Total thread wire 13.6 KiB 15.1 KiB
Claude Thread snapshot wire 7.0 KiB 7.3 KiB
Claude Live turn WebSocket wire 6.5 KiB 7.8 KiB
Claude Live turn WebSocket decoded 57.8 KiB 66.4 KiB
Claude Live turn messages 9 21

Baseline: unavailable · PR result: 5dcefb7 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 113.9 KiB
  • Claude decoded thread snapshot: 114.6 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M 30-99 changed lines (additions + deletions). vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant